workspace: Always remove removed windows

Since mutter commit 527c53a2a0582eba, MetaWorkspace::window-removed
is emitted *before* MetaWindow:workspace is updated, so the test
whether the removed window should still be on the workspace in
question will always return true.
Assume the test is no longer necessary nowadays to fix this very
obvious regression.

https://bugzilla.gnome.org/show_bug.cgi?id=735608

diff --git a/js/ui/workspace.js b/js/ui/workspace.js
index ddd043110525a7347f3bed0597a09e7449ac3a77..66058b93d0a7e87825325cf78489214b0e1290c9 100644 (file)
--- a/js/ui/workspace.js
+++ b/js/ui/workspace.js
@@ -1417,10 +1417,6 @@ const Workspace = new Lang.Class({
         if (index == -1)
             return;
 
-        // Check if window still should be here
-        if (win && this._isMyWindow(win))
-            return;
-
         let clone = this._windows[index];
 
         this._windows.splice(index, 1);

diff --git a/js/ui/workspaceThumbnail.js b/js/ui/workspaceThumbnail.js
index c11d13bd16b8100c5ef854fe1bf47d8f5e38751d..0455a9dd6a26ae4b2f714081db8249cfb2512f78 100644 (file)
--- a/js/ui/workspaceThumbnail.js
+++ b/js/ui/workspaceThumbnail.js
@@ -367,10 +367,6 @@ const WorkspaceThumbnail = new Lang.Class({
         if (index == -1)
             return;
 
-        // Check if window still should be here
-        if (win && this._isMyWindow(win) && this._isOverviewWindow(win))
-            return;
-
         let clone = this._windows[index];
         this._windows.splice(index, 1);